Skip to content

RU-T47 Fixing ios build#222

Merged
ucswift merged 1 commit intomasterfrom
develop
Feb 19, 2026
Merged

RU-T47 Fixing ios build#222
ucswift merged 1 commit intomasterfrom
develop

Conversation

@ucswift
Copy link
Member

@ucswift ucswift commented Feb 19, 2026 •
edited by coderabbitai bot
Loading

Summary by CodeRabbit

  • Chores
    • Reorganized project dependencies for build tooling optimization.

@coderabbitai
Copy link

coderabbitai bot commented Feb 19, 2026 •
edited
Loading

No actionable comments were generated in the recent review. 🎉

📝 Walkthrough

Walkthrough

The PR moves babel-plugin-transform-import-meta from devDependencies to dependencies in package.json, indicating the plugin is required at runtime rather than only during development.

Changes

Cohort / File(s) Summary
Babel Plugin Configuration
package.json		 Moved babel-plugin-transform-import-meta (^2.3.3) from devDependencies to dependencies.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • RU-T42 Fixing build issue. #215: Identical dependency migration moving babel-plugin-transform-import-meta from devDependencies to dependencies in package.json.

Suggested reviewers

  • github-actions

Poem

🐰 A tiny hop from dev to prod,
The babel plugin finds its nod,
No longer hidden in the shade,
Now at runtime, it will aid! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Title check ⚠️ Warning The title references an iOS build fix but the changeset only shows a dependency move from devDependencies to dependencies for babel-plugin-transform-import-meta, with no iOS-specific or build configuration changes evident. Update the title to accurately reflect the actual change, such as 'Move babel-plugin-transform-import-meta to production dependencies' or provide additional context in the PR description.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch develop

Tip

Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@ucswift
Copy link
Member Author

ucswift commented Feb 19, 2026

Approve

github-actions[bot]
github-actions bot approved these changes Feb 19, 2026
View reviewed changes
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This PR is approved.

@ucswift ucswift merged commit ca9198e into master Feb 19, 2026
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@github-actions github-actions[bot] github-actions[bot] approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@ucswift

Comments